Skip to content

feat(contact-center): add address book search to outdial - #565

Merged
rarajes2 merged 9 commits into
webex:nextfrom
rarajes2:outdial-address-book-CAI-7330
Dec 12, 2025
Merged

feat(contact-center): add address book search to outdial#565
rarajes2 merged 9 commits into
webex:nextfrom
rarajes2:outdial-address-book-CAI-7330

Conversation

@rarajes2

@rarajes2 rarajes2 commented Dec 2, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

COMPLETES #< CAI-7330 >

This pull request addresses

< DESCRIBE THE CONTEXT OF THE ISSUE >

Address book was missing in the OutdialCall widget. This PR adds the AddressBook feature in the OutdialCall widget with props to hide or show that part.

by making the following changes

< DESCRIBE YOUR CHANGES >

Updated the outdial widget to support address book feature. Updated all the relevant components, hooks, types etc.

Vidcast - https://app.vidcast.io/share/72f8ff38-2a30-4c69-b385-dbd0cc5a99f3

Change Type

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Tooling change
  • Internal code refactor

The following scenarios were tested

  • The testing is done with the amplify link
    < ENUMERATE TESTS PERFORMED, WHETHER MANUAL OR AUTOMATED >

Tested scenarios are shown in the vidcast

The GAI Coding Policy And Copyright Annotation Best Practices

  • GAI was not used (or, no additional notation is required)
  • Code was generated entirely by GAI
  • GAI was used to create a draft that was subsequently customized or modified
  • Coder created a draft manually that was non-substantively modified by GAI (e.g., refactoring was performed by GAI on manually written code)
  • Tool used for AI assistance (GitHub Copilot / Other - specify)
    • Github Copilot
    • Other - Please Specify
  • This PR is related to
    • Feature
    • Defect fix
    • Tech Debt
    • Automation

Checklist before merging

  • I have not skipped any automated checks
  • All existing and new tests passed
  • I have updated the testing document
  • I have tested the functionality with amplify link

Make sure to have followed the contributing guidelines before submitting.

@rarajes2
rarajes2 marked this pull request as draft December 2, 2025 17:57
@aws-amplify-us-east-2

aws-amplify-us-east-2 Bot commented Dec 2, 2025

Copy link
Copy Markdown

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-565.d1b38q61t1z947.amplifyapp.com

@rarajes2 rarajes2 added the validated Indicates that the PR is ready for actions label Dec 9, 2025
@rarajes2
rarajes2 marked this pull request as ready for review December 9, 2025 10:31
@rarajes2
rarajes2 force-pushed the outdial-address-book-CAI-7330 branch from 350d5fb to 977f861 Compare December 9, 2025 10:48
adhmenon
adhmenon reviewed Dec 11, 2025
View reviewed changes

@adhmenon adhmenon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall PR LGTM. Haven't checked snapshots but tests and code are looking good.

Comment thread packages/contact-center/task/src/task.types.ts
/**
* Flag to determine if the address book is enabled.
*/
isAddressBookEnabled: boolean;

@adhmenon adhmenon Dec 11, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just curious - as of now this is only in outdial props, but do you think we would need it in the consult-transfer popover? Right now, we just call the API and if the list is empty we don't show the tab (like agent desktop).
No need for code changes but was wondering why we have added this flag.

@rarajes2 rarajes2 Dec 12, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was a requirement for the outdial widget. For outdial we don't make any request to fetch the address book until we switch the tab, in this case if there is no contact available, it will show no contact found message. This is also similar to Agent Desktop behavior.

For consult-transfer case, everything is abstracted inside the call control widget. We can discuss if adding a prop to hide/show the address book there also makes sense.

@adhmenon adhmenon Dec 12, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Understood. Yes maybe for the popover also we can think about introducing the prop. But for now, this is good.

Comment thread packages/contact-center/cc-components/src/components/task/OutdialCall/outdial-call.tsx Outdated
Comment thread packages/contact-center/cc-components/src/components/task/OutdialCall/outdial-call.tsx
Comment thread packages/contact-center/cc-components/src/components/task/OutdialCall/outdial-call.tsx
@rarajes2
rarajes2 enabled auto-merge (squash) December 12, 2025 11:29
adhmenon
adhmenon approved these changes Dec 12, 2025
View reviewed changes

@adhmenon adhmenon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the changes. LGTM.

@rarajes2
rarajes2 merged commit 89e191c into webex:next Dec 12, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@akulakum akulakum akulakum approved these changes
@adhmenon adhmenon adhmenon approved these changes

Assignees

No one assigned

Labels

validated Indicates that the PR is ready for actions

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rarajes2 @akulakum @adhmenon